Skip to content

Prevent extra spaces in DAIDE-to-English output#50

Merged
aphedges merged 1 commit intomainfrom
fix-daide2eng-spaces
Aug 4, 2025
Merged

Prevent extra spaces in DAIDE-to-English output#50
aphedges merged 1 commit intomainfrom
fix-daide2eng-spaces

Conversation

@aphedges
Copy link
Copy Markdown
Contributor

@aphedges aphedges commented Aug 4, 2025

I noticed extra spaces before commas in the following message:

I propose an order using ENG's army in LVP to support ENG's fleet in LON moving into WAL , an order holding ENG's fleet in EDI , and an order moving ENG's fleet in LON to ECH.

There is probably a better solution, but in the interest of making minimal changes, we now remove extra spaces in and_items() and or_items().

I noticed extra spaces before commas in the following message:

> I propose an order using ENG's army in LVP to support ENG's fleet in
> LON moving into WAL , an order holding ENG's fleet in EDI , and an
> order moving ENG's fleet in LON to ECH.

There is probably a better solution, but in the interest of making
minimal changes, we now remove extra spaces in `and_items()` and
`or_items()`.
@aphedges aphedges self-assigned this Aug 4, 2025
@aphedges aphedges added the bug Something isn't working label Aug 4, 2025
@aphedges aphedges merged commit a1876fe into main Aug 4, 2025
2 checks passed
@aphedges aphedges deleted the fix-daide2eng-spaces branch August 4, 2025 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant